-
Notifications
You must be signed in to change notification settings - Fork 96
Update evictionStrategy for imported VMs #956
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request adds documentation for a known issue with imported virtual machines where the eviction strategy field is not configured during import, which prevents live migration. The documentation provides both individual VM fixes and a bulk update command for affected VMs.
Changes:
- Added a new "Eviction strategy is not set" section to the Known Issues documentation
- Provided kubectl commands to fix individual VMs and bulk update all affected VMs
- Updated documentation for versions 1.6 and 1.7
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 8 comments.
| File | Description |
|---|---|
| versioned_docs/version-v1.7/advanced/addons/vmimport.md | Added eviction strategy known issue and remediation commands for v1.7 |
| versioned_docs/version-v1.6/advanced/addons/vmimport.md | Added eviction strategy known issue and remediation commands for v1.6 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
568b15f to
111caee
Compare
jillian-maroket
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review done
The versions 1.6 and 1.7 have been updated because they are currently considered relevant. All older versions that support the VM Importer plugin are also affected, but are no longer supported anyway. Related to: harvester/harvester#9849 Signed-off-by: Volker Theile <vtheile@suse.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Jillian Maroket <67180770+jillian-maroket@users.noreply.github.com>
481a75c to
8e20caf
Compare
The versions 1.6 and 1.7 have been updated because they are currently considered relevant. All older versions that support the VM Importer plugin are also affected, but are no longer supported anyway.
Problem:
Solution:
Related Issue(s):
harvester/harvester#9849
Test plan:
Additional documentation or context